Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moderate replies #1131

Merged
merged 9 commits into from
Sep 8, 2023
Merged

Moderate replies #1131

merged 9 commits into from
Sep 8, 2023

Conversation

MatthiasReumann
Copy link
Collaborator

@MatthiasReumann MatthiasReumann commented Aug 22, 2023

Motivation and Context

Description

Add TS logic to handle reply moderation

Steps for Testing

  • 2 Account (1 Admin, 1 Studi/etc)
  • 1 Livestream with moderated chat
  1. Log in
  2. Navigate to a Livestream
  3. Reply to a message with the Studi account
  4. Approve reply with the Admin account
  5. Message should now be visible for all.

@MatthiasReumann MatthiasReumann added the enhancement New feature or request label Aug 22, 2023
@github-actions
Copy link

Your Testserver will be ready at https://1131.test.live.mm.rbg.tum.de in a few minutes.

Logins
Kurs1 Kurs2 Kurs3 Kurs4
public public loggedin enrolled
prof1 prof1 prof2 prof1
prof2
student1
student2
student3
student1
student2
student2
student3
student1
student2

@MatthiasReumann MatthiasReumann marked this pull request as ready for review August 22, 2023 10:54
Copy link
Collaborator

@alexanderstephan alexanderstephan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay, works perfectly! Nice to have this for the next semester. 👍

# Conflicts:
#	web/template/components/chat.gohtml
@alexanderstephan alexanderstephan merged commit 8170ab5 into dev Sep 8, 2023
@alexanderstephan alexanderstephan deleted the feat/moderate-replies branch September 8, 2023 10:48
SebiWrn pushed a commit that referenced this pull request May 7, 2024
Add TS logic to handle reply moderation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Moderate replies
2 participants